Skip to content

Revert "feat: Account and profile MFE legacy removal - redeployment " - #33542

Merged
abdullahwaheed merged 1 commit into
masterfrom
revert-31893-revert-31888-revert-30336-depr/account-pages
Oct 19, 2023
Merged

Revert "feat: Account and profile MFE legacy removal - redeployment "#33542
abdullahwaheed merged 1 commit into
masterfrom
revert-31893-revert-31888-revert-30336-depr/account-pages

Conversation

@abdullahwaheed

Copy link
Copy Markdown
Contributor

Reverts #31893

@abdullahwaheed
abdullahwaheed marked this pull request as ready for review October 19, 2023 07:20
@abdullahwaheed
abdullahwaheed merged commit caf8e45 into master Oct 19, 2023
@abdullahwaheed
abdullahwaheed deleted the revert-31893-revert-31888-revert-30336-depr/account-pages branch October 19, 2023 07:26
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@kdmccormick

Copy link
Copy Markdown
Member

@abdullahwaheed @Syed-Ali-Abbas-Zaidi @Mashal-m Do you folks remember why this needed to be reverted?

@jristau1984

jristau1984 commented Feb 15, 2024

Copy link
Copy Markdown
Contributor

@kdmccormick looking at NewRelic data for this deployment time, I see a spike in errors of the following:

builtins:ModuleNotFoundError
[prod-edx-edxapp-lms]
Error message: No module named 'openedx.core.djangoapps.user_api.accounts.toggles'

This error spike set off what looks like multiple alerts, which undoubtedly triggered the revert action. Sorry, but our NR data retention does not allow me to see more details about the error than what I pasted above.

I was able to dig this up from the corresponding Slack discussion:

Muhammad Abdullah Waheed Khan
[4 months ago]
actually it was failing to find module in django template, so it was causing a server error (500)

@kdmccormick

Copy link
Copy Markdown
Member

@abdullahwaheed Do you know what template it was?

@kdmccormick

Copy link
Copy Markdown
Member

I see a reference to in openedx.core.djangoapps.user_api.accounts.toggles in lms/templates/header/user_dropdown.html, but the PR removes that reference, so that shouldn't be the problem.

@Syed-Ali-Abbas-Zaidi

Copy link
Copy Markdown
Contributor

@kdmccormick, This same toggle is being used in edx-themes [ref], this was the reason why this PR got reverted. This toggle was not removed from edx-themes which caused the errors.

We also have this PR ready to remove this reference.

@Syed-Ali-Abbas-Zaidi

Copy link
Copy Markdown
Contributor

Also while digging into the issue, we found out that we need to handle all the legacy redirects of the profile and account page to redirect to the profile and account MFE instead. This PR can't go in until all the redirects are handled otherwise MFEs will break. FedBom started to create the PRs to handle the redirects but I don't know if FedBom managed to create all the PRs, @abdullahwaheed will have better knowledge about it.

Below are the PRs we created in profile and account MFE to handle the legacy redirects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants